Skip to content

ARROW-8552: [Rust] support iterate parquet row columns - #7009

Closed
houqp wants to merge 1 commit into
apache:masterfrom
houqp:master
Closed

ARROW-8552: [Rust] support iterate parquet row columns#7009
houqp wants to merge 1 commit into
apache:masterfrom
houqp:master

Conversation

@houqp

@houqp houqp commented Apr 22, 2020

Copy link
Copy Markdown
Member

This patch adds get_column_iter method to parquet rate's Row struct, which lets users iterate columns for a particular row.

@github-actions

Copy link
Copy Markdown

@houqp

houqp commented Apr 22, 2020

Copy link
Copy Markdown
Member Author

looks like the windows CI is failing with error not related to my change:

"error: \'rustfmt.exe\' is not installed for the toolchain \'nightly-2019-11-14-x86_64-pc-windows-msvc\'\nTo install

@nevi-me

nevi-me commented Apr 22, 2020

Copy link
Copy Markdown
Contributor

looks like the windows CI is failing with error not related to my change:

"error: \'rustfmt.exe\' is not installed for the toolchain \'nightly-2019-11-14-x86_64-pc-windows-msvc\'\nTo install

This will be fixed by #7010. May you please kindly rebase once it's merged

@houqp

houqp commented Apr 22, 2020

Copy link
Copy Markdown
Member Author

Thanks @nevi-me for the fix, looks like your PR has been approved. I will wait for the merge.

@houqp

houqp commented Apr 23, 2020

Copy link
Copy Markdown
Member Author

@nevi-me rebased and tests are passing now :)

Comment thread rust/parquet/src/record/api.rs Outdated
@houqp
houqp force-pushed the master branch 4 times, most recently from 0baba67 to 71ff925 Compare April 23, 2020 18:07
@houqp
houqp requested a review from andygrove April 23, 2020 18:09
@houqp
houqp force-pushed the master branch 4 times, most recently from 3ca377a to c0b3a19 Compare April 23, 2020 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants